Skip to content

diff: use the traditional timestamp in context headers in the POSIX locale - #322

Open
sylvestre wants to merge 1 commit into
diff-colorfrom
diff-context-timestamp
Open

diff: use the traditional timestamp in context headers in the POSIX locale#322
sylvestre wants to merge 1 commit into
diff-colorfrom
diff-context-timestamp

Conversation

@sylvestre

@sylvestre sylvestre commented Oct 24, 2026

Copy link
Copy Markdown
Collaborator

Stack created with GitHub Stacks CLI • Give Feedback 💬

@sylvestre
sylvestre added this pull request to stack #328 October 24, 2026 13:02
@sylvestre
sylvestre marked this pull request as ready for review October 24, 2026 13:14
Copilot AI balanced review requested due to automatic review settings October 24, 2026 13:14
Copilot AI reviewed Oct 24, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.

@sylvestre
sylvestre force-pushed the diff-context-timestamp branch from ef6e6e6 to 3576202 Compare October 24, 2026 20:13
Copilot AI balanced review requested due to automatic review settings October 24, 2026 20:13
Copilot AI reviewed Oct 24, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.

@sylvestre
sylvestre force-pushed the diff-context-timestamp branch from 3576202 to d0b291c Compare October 25, 2026 08:41
Copilot AI balanced review requested due to automatic review settings October 25, 2026 08:41
Copilot AI reviewed Oct 25, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.

@sylvestre
diff: use the traditional timestamp in context headers in the POSIX l…
d480f32 
…ocale

Should make test colors pass. cmp now also treats an empty LC_ALL or LANG as unset.
@sylvestre
sylvestre force-pushed the diff-context-timestamp branch from d0b291c to d480f32 Compare October 25, 2026 09:09
Copilot AI balanced review requested due to automatic review settings October 25, 2026 09:09
Copilot AI reviewed Oct 25, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sylvestre